Skip to content

bpo-45026: More compact range iterator (alt)#28176

Closed
serhiy-storchaka wants to merge 15 commits into
python:mainfrom
serhiy-storchaka:range-iter2
Closed

bpo-45026: More compact range iterator (alt)#28176
serhiy-storchaka wants to merge 15 commits into
python:mainfrom
serhiy-storchaka:range-iter2

Conversation

@serhiy-storchaka

@serhiy-storchaka serhiy-storchaka commented Sep 5, 2021

Copy link
Copy Markdown
Member

Keep the current and stop values instead of initial start, index and initial length.

https://bugs.python.org/issue45026

#89189

Comment thread Objects/rangeobject.c
@ambv ambv added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Sep 8, 2021
@bedevere-bot

Copy link
Copy Markdown

🤖 New build scheduled with the buildbot fleet by @ambv for commit 177da8c 🤖

If you want to schedule another build, you need to add the ":hammer: test-with-buildbots" label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Sep 8, 2021
@ambv

ambv commented Sep 10, 2021

Copy link
Copy Markdown
Contributor

The refleak here looks real:

test_range leaked [7, 7, 7] references, sum=21
test_range leaked [5, 5, 5] memory blocks, sum=15

@serhiy-storchaka

Copy link
Copy Markdown
Member Author

@ambv please do not merge until I run benchmarks.

@ambv ambv added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Sep 10, 2021
@bedevere-bot

Copy link
Copy Markdown

🤖 New build scheduled with the buildbot fleet by @ambv for commit dced0e6 🤖

If you want to schedule another build, you need to add the ":hammer: test-with-buildbots" label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Sep 10, 2021
@github-actions

Copy link
Copy Markdown

This PR is stale because it has been open for 30 days with no activity.

@github-actions github-actions Bot added the stale Stale PR or inactive for long period of time. label Oct 26, 2021
@serhiy-storchaka

Copy link
Copy Markdown
Member Author

See also #27986.

@github-actions github-actions Bot removed the stale Stale PR or inactive for long period of time. label Aug 12, 2022
@serhiy-storchaka serhiy-storchaka deleted the range-iter2 branch July 1, 2026 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting core review type-feature A feature request or enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants